Add VP8 and VP9 Android gRPC stream options - #57
Draft
krystofwoldrich-agent wants to merge 4 commits into
Draft
krystofwoldrich-agent wants to merge 4 commits into
krystofwoldrich-agent wants to merge 4 commits into
Conversation
krystofwoldrich-agent
force-pushed
the
codex/vp8-vp9-android-streaming
branch
from
September 4, 2026 11:52
9bc8af7 to
e47d8b8
Compare
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com> Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com> Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com> Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com> Co-authored-by: Codex <codex@openai.com>
krystofwoldrich-agent
force-pushed
the
codex/vp8-vp9-android-streaming
branch
from
September 4, 2026 12:11
e47d8b8 to
ab281d3
Compare
Contributor
Author
|
Post-rebase verification for feature-only head
|
krystofwoldrich
marked this pull request as draft
September 4, 2026 15:06
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Depends on expo/serve-emu#21. The submodule is pinned to feature-only commit
3f5cbc8from that branch.The independent MMAP scheduler and encoder-write cadence correction is tracked in expo/serve-emu#23 and expo-device-hub#60; it is intentionally not included here.
The pinned scrcpy 4.0 server cannot emit VP8 or VP9, so those options remain scoped to the gRPC screenshot source.
Validation
bun run typecheck— 11/11 tasks passedbun run lint— 4/4 tasks passed with no errors or warningsbun run test— 5/5 package tasks passedbun run build— 7/7 tasks passed, including vendored serve-emu and the standalone Hub bundlebun run checkab281d3is rebased on currentmainand is mergeableFeature-only 980px gRPC+MMAP comparison
These measurements were captured on the patch-equivalent pre-rebase serve-emu feature head
d5879f0with a deterministic high-motion workload, one viewer, 8 Mbps, a 60 FPS target, and 436×980 output.All codecs were constrained by the same pre-existing, codec-independent MMAP pacing ceiling. The table therefore validates comparable codec behavior rather than final 60 FPS capacity. The pipeline diagnosis, measurements, tests, and fixes live in the standalone cadence PRs linked above.
Review
The codec implementation, VP8 tuning, and the input-source integration were reviewed against the repository standards and request. Identified issues were fixed before the final rebase.